Satisfy OpenSpec finish gate for branch merge#135
Merged
Conversation
added 4 commits
April 17, 2026 15:23
Users reported gx doctor failing hard on package script mismatches even when repair could resolve them. Root cause was a duplicate legacy doctor definition overriding the active repair-first flow in the command path. This patch keeps the repair-first doctor as the only active doctor command, and aligns worktree dirty detection with normal untracked-file mode using git status --porcelain --untracked-files=normal --. Constraint: Preserve existing doctor command UX and output contract for guarded repos Rejected: Delete legacy doctor audit block entirely in this hotfix | larger refactor than needed for immediate behavior fix Confidence: high Scope-risk: narrow Reversibility: clean Directive: Do not introduce duplicate top-level command handlers with the same function name Tested: node --check bin/multiagent-safety.js; npm test; node bin/multiagent-safety.js doctor --target /home/deadpool/Documents/recodee Not-tested: npm publish/install path for this patch version
Ported the staged restore payload from the recodee restore worktree into this repository's active gx agent branch so guardex scripts, hooks, AGENTS, and OpenSpec docs are restored in one coherent changeset. Constraint: Restore source existed in sibling recodee worktree and could not be auto-finished there due elevated write limits Rejected: Recreate changes manually in-place | high risk of omission and drift Confidence: high Scope-risk: moderate Reversibility: clean Directive: Keep base and main checkout unchanged; continue restoration work only through agent worktrees Tested: bash -n on modified hook and script files Not-tested: full npm test and lint pipeline
Restored the package manifest to the guardex publish baseline and advanced version to 5.0.17, with matching README release notes, so the branch can publish a new npm artifact immediately. Constraint: NPM publish requires a non-private guardex manifest and a unique semver Rejected: Add only a version key on the restored private codex-lb-dev manifest | still not publishable Confidence: high Scope-risk: narrow Reversibility: clean Directive: Keep package.json version and README release notes in sync on every publish bump Tested: node -p package name/version; npm pack --dry-run Not-tested: npm publish to registry
Add the required branch-scoped OpenSpec checklist so agent-branch-finish can merge this release branch to main without bypassing policy guards. Constraint: agent-branch-finish enforces openspec/changes/<branch-slug>/tasks.md with sections 1-4 Rejected: bypass finish script and merge manually | violates repository merge protocol Confidence: high Scope-risk: narrow Reversibility: clean Directive: Keep branch-slug OpenSpec checklist updated before invoking finish Tested: agent-branch-finish gate precondition file structure and checklist format Not-tested: post-merge runtime behavior
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated by scripts/agent-branch-finish.sh (PR flow).